Skip to content

Conversation

johahor
Copy link
Member

@johahor johahor commented Oct 2, 2026

Added a separate .md file for specifications for the first delivery for project 2. Image still missing from details view, and we can also include some other images if we want to.

Please read through and see if there is something that is either missing or that is wrong.

Keeping it as draft until we agree on the content, and the image/images has been added.

Closes #14

@johahor
Copy link
Member Author

johahor commented Oct 2, 2026

We had not talked about some sorting options, but that is a requirement in the project. I added that in the specifications, and found that ratings is the most intuitive to be sorting on. Comments about this would be great, and also if you have different ideas for sorting.

@martakam martakam added this to the Deliverable 1 milestone Oct 2, 2026
@martakam martakam added the documentation Improvements or additions to documentation label Oct 2, 2026
martakam
martakam previously approved these changes Oct 2, 2026
Copy link
Member

@martakam martakam left a comment •

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looked good, also adjust a little bit of the information and added another sketch to illustrate the app. If you agree with all the changes, you can go ahead and merge

@martakam martakam marked this pull request as ready for review October 2, 2026 17:13
@martakam martakam requested a review from a team as a code owner October 2, 2026 17:13
Copy link
Member

@hestro hestro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good!
Updated the link to specifications.md in README

@johahor johahor merged commit c02ee93 into main Oct 2, 2026
@martakam martakam deleted the docs/specifications branch October 7, 2026 18:45
Sign in to join this conversation on GitHub.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

docs: update README with specifications for our system
3 participants